-
Notifications
You must be signed in to change notification settings - Fork 539
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Patch 1 contributing.md #896
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # app/src/main/res/raw/iso_4217_currencies.xml
…etails screen.
Increase text margins for RTL text in transaction details screen - fixes codinguser#790
Update crashlytics dependency version
Fixes add button disappearing after rotation - fixes codinguser#812
For each split in every exported transaction, the code fetched the whole Account instance, including all its transactions with all their splits. This resulted in hundreds of thousands of requests with a medium-sized ledger. There was a cache mechanism, but it was local to one single transaction, thus essentially useless. Now, each split generates two requests, to get the account's name and full name. A better cache mechanism would speed this up further.
Caching allows us to decrease the number of database requests. We did two requests per exported split before this commit; now we only do two requests per unique account in all splits.
Update to Latest commodities referenced in GnuCash desktop
Update gradle wrapper to 4.4 and plugin to 3.1.4
Initial release of croatian accounts
Fix slow CSV exports
Upgrades support library version Migrate legacy tests to use ActivityTestRule
Update Java source compatibility to Java 8 Update Travis CI build configuration
Update version for first beta release
Fix crash during export of transations after upgrade to v2.4.1-beta1 The commodities table was replaced, however, there are references to the commodities in the transactions Remove GnuCash Android Google+ Community Page link (Google+ is dead)
The commudity UIDs across the database were invalidated. A new migration has been added to refresh them after a new import. This issue only affects beta1 users. Users who went directly from v2.4.0 to v2.4.1-beta2 are not affected - Properly refrsh the book name in the accounts activity
Doublon of #884 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add few details in contributing.md (for Windows developers)